Skip to content

Conversation

GromNaN
Copy link
Member

@GromNaN GromNaN commented Apr 11, 2025

@GromNaN GromNaN requested a review from a team as a code owner April 11, 2025 11:03
@GromNaN GromNaN requested review from jmikola and removed request for a team April 11, 2025 11:03
@GromNaN GromNaN changed the base branch from v2.x to v1.21 April 11, 2025 11:03
@GromNaN GromNaN requested review from alcaeus and removed request for jmikola April 11, 2025 11:03
fi

VERSION=`$CLANG_FORMAT -version | cut -d " " -f 3`
VERSION=`$CLANG_FORMAT -version | grep -oE '[0-9]+\.[0-9]+\.[0-9]+'`
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixing this issue after update to ubuntu-24.02

/home/runner/work/mongo-php-driver/mongo-php-driver/scripts/clang-format.sh: 35: [: Illegal number: version
src/phongo_client.c:1018:14: error: code should be clang-formatted [-Wclang-format-violations]
        bson_version   = "bundled";
                    ^
src/phongo_compat.h:175:9: error: code should be clang-formatted [-Wclang-format-violations]
                return );                                     \
                      ^
src/phongo_compat.h:175:12: error: code should be clang-formatted [-Wclang-format-violations]
                return );                                     \
                         ^
make: *** [Makefile:261: format-check] Error 1

@GromNaN GromNaN merged commit 94b1dd1 into mongodb:v1.21 Apr 11, 2025
60 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants